[DO NOT MERGE] Adding InstanceFailoverGroups #2615
Conversation
Automation for azure-sdk-for-nodeNothing to generate for azure-sdk-for-node |
Automation for azure-sdk-for-pythonA PR has been created for you based on this PR content. Once this PR will be merged, content will be added to your service PR: |
Automation for azure-sdk-for-goA PR has been created for you based on this PR content. Once this PR will be merged, content will be added to your service PR: |
|
@AutorestCI rebuild azure-sdk-for-go |
Automation for azure-libraries-for-javaA PR has been created for you based on this PR content. Once this PR will be merged, content will be added to your service PR: |
|
@AutorestCI rebuild azure-sdk-for-go |
| } | ||
| ], | ||
| "properties": { | ||
| "properties": { |
There was a problem hiding this comment.
Does it need to have properties within properties?
There was a problem hiding this comment.
Isn't this right? Outer "properties" is swagger syntax, inner "properties" is the name of the actual json element.
| } | ||
| ], | ||
| "properties": { | ||
| "properties": { |
There was a problem hiding this comment.
Isn't this right? Outer "properties" is swagger syntax, inner "properties" is the name of the actual json element.
| } | ||
| } | ||
| }, | ||
| "ManagedInstancePairInfo": { |
There was a problem hiding this comment.
Can this be changed to just ManagedInstancePair ? "Info" seems redundant. All objects are information about something.
| } | ||
| } | ||
| }, | ||
| "PartnerRegionInfo": { |
There was a problem hiding this comment.
same comment about "Info" here
|
@AutorestCI regenerate azure-sdk-for-python |
This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information
api-versionin the path should match theapi-versionin the spec).Quality of Swagger